Add handling of the same-name libraries on different locations for link_nvidia_host_libraries.sh#972
Conversation
|
Instance
|
|
Instance
|
|
Instance
|
|
Instance
|
| if [[ "$target" != "/tmp/nvidia_libs/$lib"* && "$target" != *"/tmp/nvidia_libs/"* ]]; then | ||
| echo "Error: Symlink $lib_path points to $target, which is not in our mock directory" | ||
| # Verify it points to our mock library in /tmp/nvidia_libs or /tmp/nvidia_libs_duplicate | ||
| if [[ "$target" != "/tmp/nvidia_libs/$lib"* && "$target" != "/tmp/nvidia_libs_duplicate/$lib"* ]]; then |
There was a problem hiding this comment.
Shouldn't we know exactly which one it should point to? And there should be something in the output that says the other one was filtered
There was a problem hiding this comment.
No, since it is duplicate.
The real-world example was /lib was the same as /lib64 on Azure
in this case you don't care if the lib is linked from any location.
| if [ "$existing_name" = "$lib_name" ]; then | ||
| log_verbose "Duplicate library found: $lib_name (existing: $existing_lib, new: $lib_path)" | ||
| # Prioritize libraries in standard locations if possible | ||
| if [[ "$lib_path" == "/usr/lib"* || "$lib_path" == "/lib"* ]]; then |
There was a problem hiding this comment.
I'm not sure if this will catch all cases. Why not just take the first option? That is what the linker itself would do.
There was a problem hiding this comment.
Point taken, I've removed this part.
| # Continue instead of fatal_error to make the script more robust | ||
| continue |
There was a problem hiding this comment.
And what happens if the creation of all symlinks fails? I don't think this helps with robustness, to me this is an error.
There was a problem hiding this comment.
Reverted it back to fatal_error.
|
bot: build repo:eessi.io-2023.06-software arch:x86_64/amd/zen2 |
Updates by the bot instance
|
Updates by the bot instance
|
Updates by the bot instance
|
Updates by the bot instance
|
|
New job on instance
|
|
Label |
1 similar comment
|
Label |
|
I'm going to merge this, the CI is still failing due to issues with |
This is already ingested, we need to merge it.
14a9218
into
EESSI:2023.06-software.eessi.io
|
PR merged! Moved |
|
PR merged! Moved |
Check for duplicates when reading ldconfig directories.